Skip to content

Revert "CHANGE: XRI test ignored as it is causing issues in the CI. I…#2399

Merged
josepmariapujol-unity merged 3 commits intodevelopfrom
revert/input/disable-xri
Apr 7, 2026
Merged

Revert "CHANGE: XRI test ignored as it is causing issues in the CI. I…#2399
josepmariapujol-unity merged 3 commits intodevelopfrom
revert/input/disable-xri

Conversation

@josepmariapujol-unity
Copy link
Copy Markdown
Collaborator

@josepmariapujol-unity josepmariapujol-unity commented Mar 26, 2026

…t should be re-enabled once resolved (#2390)"

This reverts commit 0bb75c4.

In addition, this PR addresses the typo on the method.

Before:
XRIPackageTests.AdddingLatestXRIPackageThrowsNoErrors

After:
XRIPackageTests.AddingLatestXRIPackageThrowsNoErrors

Pass:
https://unity-ci.cds.internal.unity3d.com/job/66074078/logs 🟢

Screenshot 2026-04-07 at 10 11 33

…t should be re-enabled once resolved (#2390)"

This reverts commit 0bb75c4.
@josepmariapujol-unity josepmariapujol-unity self-assigned this Mar 26, 2026
Copy link
Copy Markdown
Contributor

@u-pr u-pr bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great
The pull request is in great shape. I found one medium-importance issue regarding event leakage in tests and a minor typo.

🤖 Helpful? 👍/👎

@codecov-github-com
Copy link
Copy Markdown

codecov-github-com bot commented Mar 26, 2026

Codecov Report

All modified and coverable lines are covered by tests ✅

@@             Coverage Diff             @@
##           develop    #2399      +/-   ##
===========================================
+ Coverage    77.92%   78.11%   +0.19%     
===========================================
  Files          482      483       +1     
  Lines        97755    98784    +1029     
===========================================
+ Hits         76175    77170     +995     
- Misses       21580    21614      +34     
Flag Coverage Δ
inputsystem_MacOS_2022.3 5.34% <ø> (-0.22%) ⬇️
inputsystem_MacOS_2022.3_project 75.39% <ø> (+0.02%) ⬆️
inputsystem_MacOS_6000.0 5.31% <ø> (-0.03%) ⬇️
inputsystem_MacOS_6000.0_project 77.28% <ø> (+0.01%) ⬆️
inputsystem_MacOS_6000.3 5.31% <ø> (-0.03%) ⬇️
inputsystem_MacOS_6000.3_project 77.28% <ø> (+0.01%) ⬆️
inputsystem_MacOS_6000.4 5.31% <ø> (-0.03%) ⬇️
inputsystem_MacOS_6000.4_project 77.29% <ø> (+0.01%) ⬆️
inputsystem_MacOS_6000.5 5.30% <ø> (-0.04%) ⬇️
inputsystem_MacOS_6000.5_project 77.32% <ø> (+0.04%) ⬆️
inputsystem_MacOS_6000.6 5.30% <ø> (-0.04%) ⬇️
inputsystem_MacOS_6000.6_project 77.32% <ø> (+0.04%) ⬆️
inputsystem_Ubuntu_2022.3_project 75.29% <ø> (+0.12%) ⬆️
inputsystem_Ubuntu_6000.0 5.31% <ø> (-0.03%) ⬇️
inputsystem_Ubuntu_6000.0_project 77.19% <ø> (+0.12%) ⬆️
inputsystem_Ubuntu_6000.3 5.31% <ø> (-0.03%) ⬇️
inputsystem_Ubuntu_6000.3_project 77.18% <ø> (+0.11%) ⬆️
inputsystem_Ubuntu_6000.4 5.32% <ø> (-0.03%) ⬇️
inputsystem_Ubuntu_6000.4_project 77.20% <ø> (+0.11%) ⬆️
inputsystem_Ubuntu_6000.5 5.31% <ø> (-0.04%) ⬇️
inputsystem_Ubuntu_6000.5_project 77.23% <ø> (+0.14%) ⬆️
inputsystem_Ubuntu_6000.6 5.31% <ø> (-0.04%) ⬇️
inputsystem_Ubuntu_6000.6_project 77.23% <ø> (+0.15%) ⬆️
inputsystem_Windows_2022.3 5.34% <ø> (-0.22%) ⬇️
inputsystem_Windows_2022.3_project 75.51% <ø> (+0.02%) ⬆️
inputsystem_Windows_6000.0 5.31% <ø> (-0.03%) ⬇️
inputsystem_Windows_6000.0_project 77.41% <ø> (+0.02%) ⬆️
inputsystem_Windows_6000.3 5.31% <ø> (-0.03%) ⬇️
inputsystem_Windows_6000.3_project 77.40% <ø> (+0.01%) ⬆️
inputsystem_Windows_6000.4 5.31% <ø> (-0.03%) ⬇️
inputsystem_Windows_6000.4_project 77.41% <ø> (+0.01%) ⬆️
inputsystem_Windows_6000.5 5.30% <ø> (-0.04%) ⬇️
inputsystem_Windows_6000.5_project 77.45% <ø> (+0.06%) ⬆️
inputsystem_Windows_6000.6 5.30% <ø> (-0.04%) ⬇️
inputsystem_Windows_6000.6_project 77.45% <ø> (+0.06%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
Assets/Tests/InputSystem.Editor/XRIPackageTest.cs 91.30% <ø> (ø)

... and 7 files with indirect coverage changes

ℹ️ Need help interpreting these results?

Copy link
Copy Markdown
Collaborator

@chris-massie chris-massie left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All tests pass now with the latest XRI version.

Fixing the typo in the method name would be nice before merge.

@josepmariapujol-unity josepmariapujol-unity merged commit b1ac60a into develop Apr 7, 2026
118 of 128 checks passed
@josepmariapujol-unity josepmariapujol-unity deleted the revert/input/disable-xri branch April 7, 2026 08:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants